Skip to content
This repository has been archived by the owner on Nov 16, 2022. It is now read-only.

Replace misaka with mistune #339

Merged
merged 6 commits into from
Sep 12, 2015
Merged

Replace misaka with mistune #339

merged 6 commits into from
Sep 12, 2015

Conversation

techtonik
Copy link
Contributor

This step makes it easier to run igp on Windows without additional Virtual Machine.

misaka.html()  ->  mistune.markdown()
misaka.EXT_STRIKETHROUGH  ->  (default)
misaka.EXT_AUTOLINK       ->  (default)
misaka.EXT_FENCED_CODE    ->  (default)
misaka.EXT_TABLES         ->  (default)

misaka.HTML_SMARTYPANTS   ->  None
  it converts (c) to copyright symbol etc.
  http://misaka.61924.nl/manual/#toc_17
  i guess it is better without such magic for now

misaka.HTML_TOC ->
  need to pack https://github.com/lepture/mistune-contrib

    git clone https://github.com/lepture/mistune-contrib
    cd mistune-contrib
    python setup.py sdist bdist_wheel --universal
    mv dist/mistune_contrib-0.1-py2.py3-none-any.whl ../vendor/mistune_contrib-b237d07-py2.py3-none-any.whl

  and add extension as described in
    https://github.com/lepture/mistune-contrib/blob/master/tests/test_toc.py
@chadwhitacre
Copy link
Contributor

Build failure?

@techtonik
Copy link
Contributor Author

@whit537 yes, old pip is unable to install stuff from vendor/ and wants to redownload it. Old virtualenv is unable to install wheels.

@techtonik
Copy link
Contributor Author

@whit537 ready to be merged.

@chadwhitacre
Copy link
Contributor

Over in H1-84740 we discovered a security configuration issue with misaka on Gratipay.com. Since we're not dealing with unvetted user input over here, we don't have the same security concern.

chadwhitacre added a commit that referenced this pull request Sep 12, 2015
Replace misaka with mistune
@chadwhitacre chadwhitacre merged commit 957ad7b into master Sep 12, 2015
@chadwhitacre chadwhitacre deleted the mistune branch September 12, 2015 22:05
@techtonik
Copy link
Contributor Author

@whit537 I checked both links with default settings for mistune - it is not vulnerable.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants